C++: Model std::vector emplace and emplace_back()#4589
Conversation
|
LGTM. Do you think it will be valuable to extend this to @dbartol in our meeting last week you asked us to try and avoid merging changes to taint tracking while you finished getting a big PR merged. Is that done, and if not is this change likely to conflict? |
|
I haven't looked closer at the other classes that we don't have support for, but at lest for Let's wait with merging this until #4432 lands, as we do touch the same test files. |
|
It looks like there are a few uses of the others, particularly |
f610831 to
e7e5754
Compare
Fixes https://github.com/github/codeql-c-analysis-team/issues/159.
I recommend commit-by-commit review for the test changes, as the addition to
stl.hlead to some churn.